Skip to content

Calculator - ready for review - #37

Closed
anathemaq wants to merge 1 commit into
the-csharp-academy:mainfrom
anathemaq:main
Closed

Calculator - ready for review#37
anathemaq wants to merge 1 commit into
the-csharp-academy:mainfrom
anathemaq:main

Conversation

@anathemaq

Copy link
Copy Markdown

Completed Calculator project for review.

Implemented:

  • Calculator with basic arithmetic operations
  • Calculation history stored in JSON
  • Ability to clear calculation history
  • Ability to use previous results from history
  • Additional operations: square root, power, multiply by 10, sin, cos, tan
  • Calculation usage counter for the current session
  • Improved console UX with Spectre.Console

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

🚀 Project Submitted!

Thanks for submitting your project! I will review it soon.

📋 Please check the dashboard to make sure the project is marked as Pending.

⚠️ If it's not marked as pending, submit it manually using the URL for this PR.

@TheCSharpAcademy

TheCSharpAcademy commented Sep 9, 2026

Copy link
Copy Markdown

@anathemaq Sorry about the delay. Project approved! 😄✅ This is a strong calculator submission with several useful extensions and a super polished Spectre.Console interface.😊

👩‍🏫Feedback

Just two very small things:

🔍️This could be expressed more cleanly with an object initializer:

calculation.Operand1 = num1;
calculation.Operand2 = num2;
calculation.Operation = op;
calculation.Result = result;

🔍️You could also eventually replace operation strings such as "a", "r" and "sin" with an enum or another strongly typed representation as the application grows.

🧐Challenges Review

✅Calculation history can be viewed in a Spectre.Console table.
✅History can be cleared.
✅Results from previous calculations can be selected and reused as either operand.
✅Power and square root operations have been added.
✅Sine, cosine and tangent are supported, with degrees correctly converted to radians before using Math.Sin, Math.Cos, and Math.Tan.
✅A session counter reports how many calculations were performed.
✅The UI has been substantially improved using Spectre.Console.


☕If you like our roadmap, please consider buying us a coffee. We appreciate your help 🙂

Overall great work!✋🏻Looking forward to seeing your next projects!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants